create intro and advanced articles for taint mode rules #2346
Conversation
✅ Don't forget to add
|
| Name | Link |
|---|---|
| 🔨 Latest commit | a2a385d |
| 🔍 Latest deploy log | https://app.netlify.com/projects/semgrep-docs-prod/deploys/68fa3738fd7f280008b7e67a |
| 😎 Deploy Preview | https://deploy-preview-2346--semgrep-docs-prod.netlify.app |
| 📱 Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify project configuration.
|
Looks great! I have one small suggestion: It’s a bit difficult to find the taint analysis docs. When you click Rule writing from the left navigation, this is what appears: My first instinct would be to expand the navigation automatically when someone enters a subsection. Since we’re not planning to do that, it might be helpful to ensure that the taint analysis docs are cross-linked from more prominent pages. |
|
@abhijna Just FYI that it isn't all or nothing when it comes to expanding the individual sections; if you take a look at sidebars.js, I changed it so that the taint analysis docs are shown if the Advanced rule-writing techniques section is open (otherwise, I think it's fine to remain hidden) |
khorne3
deleted the
katiehorne/tec-271-create-intro-and-advanced-articles-for-taint-mode-rules
branch
2 participants
This PR splits the existing taint analysis doc into two docs (one overview, one for advanced techniques). There's no new content; there are only copyedits and section reorgs plus link and anchor fixes as necessary.